Skip to content

Update opentelemetry and other dependencies #16

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Oct 31, 2025
Merged

Update opentelemetry and other dependencies #16

merged 1 commit into from
Oct 31, 2025

Conversation

@rnc
Copy link
Collaborator

@rnc rnc commented Oct 28, 2025 •
edited
Loading

Primarily this updates opentelemetry to a more recent version (thereby avoiding CVEs).

This also:

  • Updates a number of other dependencies (e.g. Quarkus (though keeping it at 2.x), SLF4J
  • Adds spotless formatting using the PNC ide-config
  • Adds redhat-releng-tools as a parent to simplify updating all the plugin versions and releasing to central.
    • This implies updating licenses and dependencyManagement to comply with parent enforcer.
  • Maintains the JDK 1.8 target (as this is used by https://github.com/project-ncl/pom-manipulation-ext )
    • Note this now needs JDK 17 to build due to spotless.
  • Updates GitHub Action versions

@vibe13 @thescouser89 Note opentelemetry-ext-cli-java is used in PME, GME, Bacon, and PNC I believe.

@rnc rnc requested a review from vibe13 October 28, 2025 10:13
@thescouser89
Copy link

thescouser89 commented Oct 28, 2025

From my quick scan, LGTM! I see some tiny refactorings for tests, and beyond that it's mostly code formatting, and version changes.

vibe13
vibe13 approved these changes Oct 30, 2025
View reviewed changes
Copy link

@vibe13 vibe13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks!

@vibe13
Copy link

vibe13 commented Oct 30, 2025

It is also used in SBOMer IIRC, to have traces in the CLI calls. Thanks @rnc !

@rnc rnc merged commit 324100b into release-engineering:main Oct 31, 2025
2 checks passed
@rnc rnc deleted the BR1 branch October 31, 2025 08:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vibe13 vibe13 vibe13 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@rnc @thescouser89 @vibe13